feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/step/ImpressionStep.kt (4)

73-77: 하드코딩된 150ms 지연 대신 프레임/인셋 신호 기반으로 전환 고려

지연값을 100→150ms로 늘린 것은 증상 완화에 도움이 되지만, IME 애니메이션 시간은 OEM/IME마다 상이해 하드코딩 지연은 여전히 간헐적 실패 여지가 있습니다. 프레임 동기화 또는 인셋 가시화 신호를 활용하면 더 견고합니다. 최소한 지연값을 의미 있는 상수로 추출해 관리 가능성을 높이는 것을 권장합니다.

• 옵션 A(간단): 의미 있는 상수로 추출

-            delay(150)
+            delay(IME_RELOCATION_DELAY_MS)

아래 상수 선언을 파일 상단(함수 바깥) 등에 추가:

private const val IME_RELOCATION_DELAY_MS = 150L

• 옵션 B(권장): 프레임 1~2회 동기화 후 호출

-            delay(150)
-            bringIntoViewRequester.bringIntoView()
+            withFrameNanos { /* 1st frame after IME starts animating */ }
+            withFrameNanos { /* 2nd frame to ensure insets are applied */ }
+            bringIntoViewRequester.bringIntoView()

검증 제안:

• 삼성/픽셀/샤오미 등 주요 OEM, 제스처/3버튼 내비게이션, 세로/가로, 다양한 IME(네이버/구글/삼성)에서 가이드 버튼이 매번 1st-try로 가려지지 않는지 확인 바랍니다.

---

87-93: 하단 패딩 16.dp → 디자인 토큰 사용으로 일관성 유지 제안

하단 여백을 늘린 조정은 긍정적입니다. 다만 하드코딩된 16.dp 대신 테마 토큰 사용을 권장합니다(디자인 일관성, 다크/다국어/디바이스 스케일 대응 용이).

-                .padding(bottom = 16.dp)
+                .padding(bottom = ReedTheme.spacing.spacing4) // spacing4가 16dp와 동일한지 확인

Note: spacing4가 16dp가 아닐 경우, 동일 값에 해당하는 토큰으로 교체 부탁드립니다.

---

31-33: 현재 포커싱 자동 동작 제거에 따라 FocusRequester는 미사용 — 정리 권장

자동 포커싱/키보드 표시 로직을 제거하면서 FocusRequester를 생성/부착만 하고 실제로 사용(예: requestFocus())하지 않습니다. 혼선을 줄이기 위해 관련 코드를 정리하면 좋겠습니다.

- import androidx.compose.ui.focus.FocusRequester
- import androidx.compose.ui.focus.focusRequester
  import androidx.compose.ui.focus.onFocusChanged

-    val focusRequester = remember { FocusRequester() }

                 modifier = Modifier
                     .fillMaxWidth()
-                    .focusRequester(focusRequester)
                     .height(140.dp)
                     .onFocusChanged { focusState ->
                         isImpressionTextFieldFocused = focusState.isFocused
                     },

Also applies to: 67-68, 109-115

---

143-146: leadingIcon은 보조적 요소이므로 contentDescription을 null로 설정 권장

버튼의 접근성 라벨은 텍스트로 충분히 전달됩니다. 아이콘에 별도 contentDescription을 주면 스크린리더에서 중복 낭독될 수 있습니다. 장식 목적이면 null이 바람직하고, 별도 설명이 필요하다면 stringResource로 현지화된 문자열을 사용해 주세요.

-                            contentDescription = "Impression Guide Icon",
+                            contentDescription = null,

대안(현지화 필요 시):

contentDescription = stringResource(R.string.impression_guide_icon_description)

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

Learnt from: seoyoon513
PR: YAPP-Github/Reed-Android#46
File: feature/search/src/main/kotlin/com/ninecraft/booket/feature/search/component/InfiniteLazyColumn.kt:83-95
Timestamp: 2025-07-14T00:46:03.843Z
Learning: seoyoon513과 팀은 한국어 주석을 선호하며, 한국어 주석을 영어로 번역하라는 제안을 하지 않아야 함

feature/record/src/main/kotlin/com/ninecraft/booket/feature/record/step/ImpressionStep.kt (1)

59-72: PR 목표 달성 확인: 자동 키보드 표시 제거 및 가려짐 완화 흐름 정돈

• 초기 자동 포커스/키보드 표시 제거로 진입 시 레이아웃 밀림 이슈가 해소된 점 👍
• 키보드 가시화 + 텍스트필드 포커스일 때 가이드 버튼을 bringIntoView로 보장하는 흐름이 PR 의도와 부합합니다.